Skip to content

feat: security management group#106

Merged
jaredfholgate merged 3 commits into
mainfrom
feat--security-management-group
Sep 12, 2025
Merged

feat: security management group#106
jaredfholgate merged 3 commits into
mainfrom
feat--security-management-group

Conversation

@jaredfholgate

@jaredfholgate jaredfholgate commented Sep 10, 2025 •
edited
Loading

Copy link
Copy Markdown
Member

Overview/Summary

Adding security management group and subscription.

Bug fix update to hub networking module

This PR fixes/adds/changes/removes

N/A

Breaking Changes

Adds a 4 subscription and a new management group

Testing Evidence

Please provide any testing evidence to show that your Pull Request works/fixes as described and planned (include screenshots, if appropriate).

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

matt-FFFFFF
matt-FFFFFF reviewed Sep 11, 2025
View reviewed changes

@matt-FFFFFF matt-FFFFFF left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one thing to consider

Comment thread alz/local/variables.tf
@jaredfholgate

jaredfholgate commented Sep 11, 2025 •
edited
Loading

Copy link
Copy Markdown
Member Author

E2E test run: https://github.com/Azure/accelerator-bootstrap-modules/actions/runs/17650198444/job/50158928371

@jaredfholgate jaredfholgate self-assigned this Sep 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jtracey93 jtracey93 jtracey93 approved these changes
+1 more reviewer
@matt-FFFFFF matt-FFFFFF matt-FFFFFF left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@jaredfholgate jaredfholgate

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jaredfholgate @matt-FFFFFF @jtracey93